chore: upgrade ng from 11 to 12#302
Conversation
This comment has been minimized.
This comment has been minimized.
Codecov Report
@@ Coverage Diff @@
## main #302 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 49 49
Lines 655 655
Branches 62 62
=========================================
Hits 655 655 Continue to review full report at Codecov.
|
| "@angular/forms": "^12.2.1", | ||
| "@angular/platform-browser": "^12.2.1", | ||
| "@angular/platform-browser-dynamic": "^12.2.1", | ||
| "@angular/router": "^12.2.1", |
There was a problem hiding this comment.
what about zone.js? does it need to be upgraded?
There was a problem hiding this comment.
No, I think we're already on at latest one https://www.npmjs.com/package/zone.js?activeTab=versions
| * on performance if an error is thrown. | ||
| */ | ||
| // import 'zone.js/dist/zone-error'; // Included with Angular CLI. | ||
| // import 'zone.js/plugins/zone-error'; // Included with Angular CLI. |
There was a problem hiding this comment.
I see this change here but no change in zone.js dependency. Is this right?
There was a problem hiding this comment.
Yeah, answered above.
There was a problem hiding this comment.
above you answered that zone.js doesn't need to be upgraded - but I think anand is asking: then why would this change? was it previously wrong?
It looks like the zone docs agree that it was:
https://github.com/angular/angular/tree/master/packages/zone.js#bundles
|
@itssharmasandeep quick question: What is peer dependency in npm? |
| * on performance if an error is thrown. | ||
| */ | ||
| // import 'zone.js/dist/zone-error'; // Included with Angular CLI. | ||
| // import 'zone.js/plugins/zone-error'; // Included with Angular CLI. |
There was a problem hiding this comment.
above you answered that zone.js doesn't need to be upgraded - but I think anand is asking: then why would this change? was it previously wrong?
It looks like the zone docs agree that it was:
https://github.com/angular/angular/tree/master/packages/zone.js#bundles
|
We need to upgrade peer dependency before merging this pr. i have shared reading material with sandeep on slack |
Co-authored-by: Aaron Steinfeld <45047841+aaron-steinfeld@users.noreply.github.com>
This comment has been minimized.
This comment has been minimized.
right - forgot this repo even had the separate project |
|
taking |
Unit Test Results 2 files 15 suites 24s ⏱️ Results for commit 84b0544. |
why? without the peer dependencies this isn't usable (so shouldn't have been merged). what am I missing? |
|
@aaron-steinfeld I want sandeep to try out and learn the entire process. Explained him few things, but a good hands on would help him learn the entire flow. So, I asked him to merge this PR and see what is breaking in ht-ui when he tries to add the new bundle. |
|
🎉 This PR is included in version 2.6.0 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
Due to some problem with #297
This is the new one with addressed comments